Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Фикс работы с температурой консольной команды setmapatmos #2589

Conversation

Fildrance
Copy link
Contributor

@Fildrance Fildrance commented Sep 13, 2024

Описание PR

Теперь команда setmapatmos может выставлять атмосфере температуру из переданных аргументов.

Почему / Баланс

Ранее команда выставляла при вызове атмосу температуру космоса по-умолчанию - 2.7F (даже если команда вызывалась на планетарной карте).

Технические детали

1-line commit

Медиа

fix_setmapatmos_command.webm

Требования

Критические изменения

нет
Список изменений

@Fildrance
Copy link
Contributor Author

Список изменений не пишу - изменение для администраторов, а не для игроков

@Fildrance
Copy link
Contributor Author

Желающим воспользоваться - просьба учитывать, что команда меняет атмосферу "на карте", но не на гриде самой станции (то есть на планете за рамками станции, то есть у входов в отделы температура и атмосфера поменяются не мгновенно а за счет процесса смешивания атмосферы и обмены температуры).

@Fildrance Fildrance marked this pull request as ready for review September 13, 2024 16:02
@github-actions github-actions bot added the Status: Needs Review This PR requires new reviews before it can be merged label Sep 13, 2024
@Fildrance
Copy link
Contributor Author

Отдельно фикс будет подготовлен в репо wizden, данный PR сделан для того, чтобы использовать команду на ивенте Соляриса 15-го сентября, поэтому PR именно в corvax-репозиторий, а не сначала в wizden а потом уже ожидание пока его туда примут и upstream прольется.

@Fildrance
Copy link
Contributor Author

Судя по PR в визден - space-wizards/space-station-14#32125 тест флаки, надо перезапустить.

@Fildrance Fildrance closed this Sep 15, 2024
@Fildrance Fildrance deleted the fix/invalid-temp-in-set-map-atmos-command branch September 15, 2024 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant